Skip to content

Conversation

kartben
Copy link
Contributor

@kartben kartben commented Sep 16, 2025

A follow-up to commit d6ce2f4 that introduced a new fakedeferdriver node in app.overlay, but due to how overlays are applied, the board specific overlay files also need to be updated to avoid build errors, as they override whatever app.overlay has.

Fixes a few errors caught in weekly CI

  • "hifive_unmatched/fu740/s7:kernel.device",
  • "hifive_unmatched/fu740/s7:kernel.device.minimallibc",
  • "hifive_unmatched/fu740/s7:kernel.device.pm",
  • "hifive_unmatched/fu740/u74:kernel.device",
  • "hifive_unmatched/fu740/u74:kernel.device.minimallibc",
  • "hifive_unmatched/fu740/u74:kernel.device.pm"

cc @ljd42

peter-mitsis
peter-mitsis previously approved these changes Sep 16, 2025
@kartben
Copy link
Contributor Author

kartben commented Sep 19, 2025

ping @tbursztyka, this will help get less failures in weekly CI - thanks!

Copy link
Contributor

@ljd42 ljd42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls update the reg property.

A follow-up to commit d6ce2f4 that
introduced a new fakedeferdriver node in app.overlay, but due to how
overlays are applied, the board specific overlay files also need to
be updated to avoid build errors, as they override whatever app.overlay
has.

Signed-off-by: Benjamin Cabé <[email protected]>
Copy link

@ljd42
Copy link
Contributor

ljd42 commented Sep 19, 2025

Changes is fine to fix the CI issues.

In a longer run, we should refactor the overlays to have a "DRYer" solution. and avoid having to update overlays in multiple places.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants